Skip to content

Fix main CI: drop unused import (shake 39→38)#73

Merged
Xinze-Li-Moqian merged 5 commits into
mainfrom
develop
Jun 14, 2026
Merged

Fix main CI: drop unused import (shake 39→38)#73
Xinze-Li-Moqian merged 5 commits into
mainfrom
develop

Conversation

@Xinze-Li-Moqian

Copy link
Copy Markdown
Contributor

main 上 #72 合并后 CI 红:Shake unused-import 检查 39 > 38 基线。新 Manifold 域的 CoordinateBall.lean 带了一个未用 import Mathlib.Geometry.Manifold.ContMDiff.Atlas(原移植带来的),删之即回到 38。

本 PR 同时带上 develop 上自 #72 以来的内容(#66 precompact-basis + 台账更新)。已本地验证:全库构建 3670 jobs + shake = 38。

LehengChen and others added 5 commits June 14, 2026 15:04
…refinement

Port Lee Lemma 1.10 (countable precompact-coordinate-ball basis) and
Theorem 1.15 (countable locally finite refinement subordinate to any open
cover) into OpenGALib/Manifold/Charts/PrecompactBasis.lean.

The source binder [TopologicalManifold n M] is restated as the four Mathlib
classes [TopologicalSpace M] [T2Space M] [SecondCountableTopology M]
[ChartedSpace (EuclideanSpace ℝ (Fin n)) M]; the
TopologicalManifold.locallyCompactSpace_of_topologicalManifold projection is
re-pointed to ChartedSpace.locallyCompactSpace.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Single **Math.** anchor tags (dropped **Eng.** proof-narration), module
docstring re-framed by architectural layer (basis needs only topology +
chart; refinement adds paracompactness via a reusable abstract lemma),
provenance footer. Statements and proofs unchanged.
Precompact coordinate-ball countable basis + locally-finite refinement.
Original port by LehengChen (#66); docstrings revised to house style.
Builds on #65 coordinate-ball foundations.
Mathlib.Geometry.Manifold.ContMDiff.Atlas was unused; shake baseline is
38 and the new domain pushed it to 39, failing CI on the #72 main merge.
Carried over from the original port; build + linters unaffected.
@Xinze-Li-Moqian Xinze-Li-Moqian merged commit 2e3b63a into main Jun 14, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants